Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Modification for showing error messages with destination 'popup' #54

Closed
wants to merge 1 commit into from

Conversation

sribo23
Copy link

@sribo23 sribo23 commented Aug 25, 2016

Modification for showing error messages with destination 'popup' although the theme option 'Select action when product is added to cart' is set to 'Open popup'.

…ough the theme option 'Select action when product is added to cart' is set to 'Open popup'.
@hkreuter
Copy link
Contributor

hkreuter commented Dec 7, 2016

Hi sribo23,
the issue you try to fix in this PR is related to https://bugs.oxid-esales.com/view.php?id=6479. It might be better to show the error message in the popup, what do you think?

@kermie
Copy link
Contributor

kermie commented Feb 27, 2017

@sribo23 could you please sign the CLI as it is mandatory for accepting a pull request. Cheers!

@Sieg Sieg self-assigned this Nov 22, 2017
Sieg added a commit that referenced this pull request Nov 22, 2017
@Sieg
Copy link
Member

Sieg commented Nov 22, 2017

I have added some change to show those "popup" errors in modal box. Please check if i haven't missed anything.

@Sieg
Copy link
Member

Sieg commented Dec 15, 2017

Hey @sribo23, the CLA is not signed for this repository. Please check the Details link down in github Check box.

@Sieg
Copy link
Member

Sieg commented Apr 27, 2018

Merged together with my improved variant.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants